Skip to content
This repository has been archived by the owner on Jan 13, 2023. It is now read-only.

get_bundles can return multiple bundles #293

Merged
merged 1 commit into from
Jan 27, 2020

Conversation

lzpap
Copy link
Member

@lzpap lzpap commented Jan 23, 2020

Solves: #250

Changes

  • get_bundles extended API call accepts a list of tail transaction hashes instead of just one.
  • get_bundles returns multiple bundles in the same order as the input tail hashes.
  • Added test coverage for returning multiple bundles.
  • Modified calls to GetBundlesCommand() throughout the lib.

WARNING: Breaking change!

Copy link
Contributor

@todofixthis todofixthis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Strongly recommend changing GetBundlesRequestFilter and adding corresponding test; otherwise looks good with minor questions/suggestions.

iota/commands/extended/get_bundles.py Outdated Show resolved Hide resolved
iota/commands/extended/get_bundles.py Outdated Show resolved Hide resolved
test/commands/extended/get_bundles_test.py Outdated Show resolved Hide resolved
test/commands/extended/get_bundles_test.py Show resolved Hide resolved
test/commands/extended/get_bundles_test.py Show resolved Hide resolved
Copy link
Member Author

@lzpap lzpap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thx for the review @todofixthis !

test/commands/extended/get_bundles_test.py Show resolved Hide resolved
test/commands/extended/get_bundles_test.py Show resolved Hide resolved
Copy link
Contributor

@todofixthis todofixthis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for making those changes! Looks good (with one minor suggestion 😁)!

test/commands/extended/get_bundles_test.py Outdated Show resolved Hide resolved
- accept a list of tail transaction hashes instead
  of just one.
- return multiple bundles in the same order as
  the input tail hashes
- added test coverage
- modified calls to GetBundlesCommand() in the lib

WARNING: Breaking change!
@lzpap lzpap merged commit 04fb2d0 into iotaledger:develop Jan 27, 2020
@lzpap lzpap deleted the multiple_bundles branch January 27, 2020 09:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants